Skip to content

refactor: resolve the site name and addresses in one place - #528

Merged
has2k1 merged 1 commit into
mainfrom
refactor/identity
Nov 4, 2026
Merged

has2k1 merged 1 commit into
mainfrom
refactor/identity

Conversation

@has2k1

@has2k1 has2k1 commented Nov 4, 2026

Copy link
Copy Markdown
Collaborator

This PR gathers the site name, the configured site URL and the preferred address for canonical links into one value, built from the configuration, the repository and the generated _quarto.yml. Page titles, social cards, structured data, raw custom pages and great-docs seo used to ask the build object for each value through separate methods, each of which detected the repository again. Copying the default social card image into the site now lives with the rest of the social card code. No output changes.

Page titles, social cards, structured data and SEO output now share the
site name and URL resolution rules. The site name still prefers
`website.title` in the generated `_quarto.yml`, then the configured
display name, then the package name. Canonical URLs still prefer
`seo.canonical.base_url`, then `site_url`, then the repository's GitHub
Pages address.

Copying the default social card image into the site and addressing it
now lives with the other social card code. It still warns about a
missing local image, and builds, raw custom pages and the `seo` command
produce the same output as before.
@github-actions

github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown

📚 Preview this PR's docs locally

Fetch the just-built site and open it in your browser:

# this exact build
great-docs preview --run 41811061009

# or always the latest build for this PR
great-docs preview --pr 528
Auth & tips
  • Requires great-docs: pip install great-docs (or run via pipx/uvx).
  • Auth: run gh auth login then add --use-gh, or set GITHUB_TOKEN (needs Actions: read).
  • Jump to a page: add --path reference/index.html.
  • Latest vs exact: --pr <n> grabs the newest build; --run <id> pins this one.
  • Re-download: --refresh ignores the local cache.

@has2k1
has2k1 merged commit 9e0c0c9 into main Nov 4, 2026
9 checks passed
@has2k1
has2k1 deleted the refactor/identity branch November 4, 2026 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant